-
Notifications
You must be signed in to change notification settings - Fork 7
docs: updates architecture diagram and test instructions in README #70
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Codecov Report
@@ Coverage Diff @@
## main #70 +/- ##
=========================================
Coverage 56.26% 56.26%
Complexity 41 41
=========================================
Files 20 20
Lines 375 375
Branches 18 18
=========================================
Hits 211 211
Misses 160 160
Partials 4 4
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
|
||
| To test your image using the docker-compose setup follow the steps: | ||
|
|
||
| - Commit you changes to a branch say `graphql-service-test`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: We describe how to test via docker compose in a section called "Testing image" and via helm in one called "Helm setup". I think It would make more sense to break it down into two sections under "Testing image" like "With docker-compose" and "With helm"
Co-authored-by: Aaron Steinfeld <45047841+aaron-steinfeld@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
* Added config for gateway gRPC client deadline * Spotless apply * Changed gateway-service client timeout unit to java.time.Duration and added fallback configuration
Description
This PR,
Testing
NA
Checklist:
Documentation